Skip to content

[stable35] fix(comments): recreate comment editor on Activity tab remount - #64079

Merged
AndyScherzinger merged 2 commits into
stable35from
backport/63505/stable35
Sep 7, 2026
Merged

[stable35] fix(comments): recreate comment editor on Activity tab remount#64079
AndyScherzinger merged 2 commits into
stable35from
backport/63505/stable35

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 5, 2026

Copy link
Copy Markdown

Backport of PR #63505

@backportbot
backportbot Bot requested a review from a team as a code owner September 5, 2026 12:18
@backportbot
backportbot Bot requested review from joshtrichards, jospoortvliet, kristian-zendato, skjnldsv, sorbaugh and szaimen and removed request for a team September 5, 2026 12:18
@backportbot backportbot Bot added bug 3. to review Waiting for reviews papercut Annoying recurring UX issue with possibly simple fix. feature: comments feature: file sidebar Related to the file sidebar component AI assisted labels Sep 5, 2026
@backportbot backportbot Bot added this to the Nextcloud 35 milestone Sep 5, 2026

@szaimen szaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

…ment editor

The comment editor Vue app instance was kept in a closure variable and
reused across Activity tab mount/unmount cycles. unmount() destroyed
the app via app.unmount() but never reset the variable, so the next
mount() skipped creating a new app and called mount() on an already-
destroyed instance — a silent no-op. This made the comment field
disappear after leaving and returning to the Activity tab, only
recoverable via a full page reload.

Reset `app` to undefined after unmounting so the editor is recreated
on the next mount.

Fixes #63487

Assisted-by: Claude:claude-sonnet-5

Signed-off-by: Jos Poortvliet <jospoortvliet@gmail.com>
@AndyScherzinger

Copy link
Copy Markdown
Member

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger
AndyScherzinger merged commit 3a10200 into stable35 Sep 7, 2026
83 of 84 checks passed
@AndyScherzinger
AndyScherzinger deleted the backport/63505/stable35 branch September 7, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted bug feature: comments feature: file sidebar Related to the file sidebar component papercut Annoying recurring UX issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants